Feedback addressed:

• Fixed "off-chain" → "offchain" per project spelling convention (line 9)

Feedback addressed:

• Added note in "Testing locally" section about single-node consensus: transform functions aren't exercised locally, developers should verify transforms handle varying inputs before deploying to multi-node subnets

Feedback addressed:

• Fixed "Both GET, HEAD, and POST" → "The GET, HEAD, and POST" (grammar)
• Removed unused serde_json from Cargo.toml dependencies
• Added Rust function name hint (get_icp_price) to testing section CLI example

Review: HTTPS Outcalls

Must fix

• Missing exchange-rates example link: The content brief explicitly says "Link to the exchange-rates example for a complete real-world use case." The page links to send_http_get and send_http_post but omits the exchange-rates example, which demonstrates a real production pattern (multi-canister, periodic data fetching). Add it — it's the most valuable example for developers building real applications.

• Response size limit (2MB) not called out prominently: The content brief says "Cover... response size limits." The page only mentions this indirectly in the cycle costs section ("assumes 2MB"). The 2MB hard limit is a common surprise for developers — it deserves its own sentence or bullet, e.g., "The maximum response body is 2MB (2,097,152 bytes). Requests exceeding this limit fail." The icskill lists this as mistake docs: application architecture concept page #4 that breaks builds.

• POST example snippet is not self-contained: The Motoko POST snippet uses Blob, Text, IC, and Call without imports, and transformPost/postData appear outside any actor. A reader copying this snippet gets a compile error. Either wrap it in a full actor with imports (like the GET example) or add a note like "Add these methods inside the actor from the GET example above."

Suggestions

• No Rust POST example: The POST section only shows Motoko. The GET section has both Rust and Motoko. For consistency and because Rust is the more common IC language, consider adding a Rust POST snippet or at minimum linking to the Rust send_http_post example and the Motoko send_http_post example (currently only links to the Rust version).

• HEAD method not mentioned in body: The intro says "The GET, HEAD, and POST methods are supported" but HEAD is never shown or discussed. Either drop it from the intro or add a one-liner explaining when HEAD is useful (e.g., checking if a resource exists without downloading the body).

• "Host" header pitfall omitted: The icskill lists "Forgetting the Host header" as a common mistake — some API endpoints require it explicitly. Worth a one-line note in the transform or GET section.

• Localhost/private IP restriction not mentioned: The icskill notes that HTTPS outcalls can only reach public internet endpoints — localhost, 10.x.x.x, 192.168.x.x are blocked. This is a common gotcha for developers testing against local services.

• is_replicated field unexplained: Both code examples set is_replicated = null/None without explaining what it does. Either drop it (since None is the default) or add a brief comment explaining it controls whether the outcall goes through consensus.

Verified

• All internal links resolve: ../../reference/management-canister.md, ../../concepts/https-outcalls.md, ../chain-fusion/ethereum.md, ../../reference/cycles-costs.md all exist
• External URLs verified: github.com/dfinity/examples/tree/master/rust/send_http_get, motoko/send_http_get, rust/send_http_post all match .sources/examples/
• CLI commands verified against .sources/icp-cli/docs/reference/cli.md: icp network start -d, icp deploy backend, icp canister call are all valid
• No dfx references — clean
• No .mdx/JSX — plain markdown throughout
• Motoko uses mo:core/ (not mo:base/) per content rules
• Rust imports and API patterns (ic_cdk::management_canister::http_request, TransformArgs, TransformContext, TransformFunc) match icskill and .sources/examples/
• Call.httpRequest (Motoko auto-cycle-attachment) verified against icskill
• Cycle cost formula matches icskill reference (49M base, 5200/request byte, 10400/response byte on 13-node subnet)
• Frontmatter complete and consistent with body
• Sync recommendation present
• Previous feedback (offchain spelling, local testing note, grammar fix, unused serde_json removal) appears addressed per PR comments

Feedback addressed:

Must-fix:

• Added 2MB response limit as prominent bullet (docs: network overview concept page #3) in "How HTTPS outcalls work" section
• Added note that Motoko POST snippet goes inside the GET actor above (self-contained context)
• Added Exchange Rate Canister (XRC) link in "What's next" as a production service powered by HTTPS outcalls

Suggestions:

• Added both Rust and Motoko links for send_http_post example
• Added HEAD method explanation in the intro paragraph
• Added "Limitations and pitfalls" section: public endpoints only, Host header requirement, ~30s timeout
• Added non-replicated (flexible) outcalls note as future work (tracks portal#5890)
• Removed is_replicated from code examples (None is the default; avoids unexplained field)